ALICE3-TRK: several fixes in the digitization code#14733
Merged
mconcas merged 5 commits intoAliceO2Group:devfrom Oct 13, 2025
Merged
ALICE3-TRK: several fixes in the digitization code#14733mconcas merged 5 commits intoAliceO2Group:devfrom
mconcas merged 5 commits intoAliceO2Group:devfrom
Conversation
Contributor
atriolo
commented
Oct 13, 2025
atriolo
commented
- Deal with ML/OT L4 mixed length and shape when assigning row and columns to the digit
- Hardcoded ML/OT staves length according to the current geometry
- Number of steps to calculate the response increased to 25 due to the larger silicon thickness in the geometry
- Charge threshold to collect the signal temporarily removed: no reason to put an arbitrary value
- Considering the OT half-stave as the smallest element for the digitization in the OT
- Wrong shape of digit distributions along columns fixed: due to the improper rescaling of the depth of the sensor when calculating the response of the pixel
…orarly the threshold to 1 electron
…caling of the silicon depth
…or the digitization in the OT
Contributor
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
mconcas
approved these changes
Oct 13, 2025
njacazio
approved these changes
Oct 13, 2025
plariono
pushed a commit
to plariono/AliceO2
that referenced
this pull request
Oct 23, 2025
* ALICE3-TRK: deal with ML/OT L4 mixed lenght and shape * ALICE3-TRK: adjust ML/OT staves length according to the current geometry * ALICE3-TRK: increase number of steps into silicon to 25. Setting temporarly the threshold to 1 electron * ALICE3-TRK: fix digits distribution along columns by removing the rescaling of the silicon depth * ALICE3-TRK: considering the OT half-stave as the smallest elemenent for the digitization in the OT
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.